Skip to content

Conversation

@mattgodbolt
Copy link
Member

@mattgodbolt mattgodbolt merged commit a99d06e into main Dec 20, 2024
@mattgodbolt mattgodbolt deleted the mg/clad branch December 20, 2024 14:49
fi

URL=https://github.com/vgvassilev/clad
CLANG_VERSION=18.1.0
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would make this configurable and include it in the S3 name so we can support the plugin for multiple clang versions

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll take a look at that.

@Vipul-Cariappa
Copy link
Contributor

Hi. We were thinking about adding a nightly build of clad to godbolt.
I see that the build script already supports that #101.
I know, we will need to update similar to compiler-explorer/compiler-explorer#7109 with the trunk version.
However, I am unsure what/where changes are required to configure the CI for a nightly build. I am guessing we will need to add

nightly:
  if: nightly
  type: nightly
  targets:
    - trunk

to compiler-explorer/infra#1484 ?


We could also move clad to a separate compiler group with this.

@mattgodbolt
Copy link
Member Author

Something like that yes: if you look at the clangs or similar you'll see how they do it.

We'd also need it to be added to our "daily builds" -> https://github.com/compiler-explorer/compiler-workflows - I can help with either of these.

@Vipul-Cariappa
Copy link
Contributor

Hi. I have opened PRs at compiler-explorer/compiler-explorer#7311 and compiler-explorer/compiler-workflows#31 to add the nightly build of clad.
I am not sure about the changes required in https://github.com/compiler-explorer/infra.

@mattgodbolt
Copy link
Member Author

First run: https://github.com/compiler-explorer/compiler-workflows/actions/runs/12993976934

Changes in infra; just adding a "nightly" equivalent build (see one of the other nightly compilers). I can help if you can't make any sense of the setup (it's not well documented)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants